Skip to content

Fix: side bar overflowing - #8322

Merged
avivkeller merged 1 commit into
webpack:mainfrom
Pranav-IIITM:fix/ad-sidebar-overflow
Aug 9, 2026
Merged

Fix: side bar overflowing#8322
avivkeller merged 1 commit into
webpack:mainfrom
Pranav-IIITM:fix/ad-sidebar-overflow

Conversation

@Pranav-IIITM

Copy link
Copy Markdown
Contributor

Summary

Closes #8321

This PR fixes a layout bug where the left-hand Sponsor sidebar would overflow the viewport on 100% Zoom

Changes made:

can be seen in the screenshot

Before After
image image

What kind of change does this PR introduce?

Fix (CSS/layout bug fix).

Did you add tests for your changes?

No automated tests were added. This was verified through manual visual testing at multiple screen sizes and zoom levels (see below).

Does this PR introduce a breaking change?

No.

If relevant, what needs to be documented once your changes are merged or what have you already documented?

No additional documentation is needed — this is a visual/layout fix with no API or usage changes.

Use of AI

Not used

@vercel

vercel Bot commented Aug 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
webpack-js-org Ready Ready Preview Aug 8, 2026 11:51pm

Request Review

@linux-foundation-easycla

linux-foundation-easycla Bot commented Aug 8, 2026

Copy link
Copy Markdown

CLA Signed
The committers listed above are authorized under a signed CLA.

  • ✅ login: Pranav-IIITM / name: Pranav-IIITM (7465cdf)

@Pranav-IIITM Pranav-IIITM changed the title fiexed side bar overflowing Fix: side bar overflowing Aug 8, 2026
@evenstensberg

Copy link
Copy Markdown
Member

Hi, thanks for your contribution! Could you sign the CLA?

Signed-off-by: Pranav-IIITM <jogdandpranav2007@gmail.com>
@Pranav-IIITM

Copy link
Copy Markdown
Contributor Author

@evenstensberg I have signed the CLA and all checks have passed too !

@evenstensberg evenstensberg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@Pranav-IIITM

Copy link
Copy Markdown
Contributor Author

Hi @evenstensberg, thanks for the review! While working on this fix I noticed a related improvement that could be made (a feature request rather than a bug fix). Should I go ahead and open a new issue for that separately, even though this PR is still open/unmerged? Just want to make sure I'm following the right process. Thanks!

@avivkeller
avivkeller merged commit 96ab0dd into webpack:main Aug 9, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UI Bug: Sponsor banners overflow and break sidebar layout on Desktop

3 participants